DOC-17056: Remove cockroach issue links from known limitations#23302
Open
ebembi-crdb wants to merge 2 commits into
Open
DOC-17056: Remove cockroach issue links from known limitations#23302ebembi-crdb wants to merge 2 commits into
ebembi-crdb wants to merge 2 commits into
Conversation
Unlink 542 known limitation files by converting issue references like [#96398](cockroachdb/cockroach#96398) to plain #96398 text. The issue numbers are preserved for reference. Part of DOC-17056. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Files changed:
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
4 tasks
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
rmloveland
requested changes
May 21, 2026
Contributor
rmloveland
left a comment
rmloveland
left a comment
There was a problem hiding this comment.
please remove all the dangling "tracking github issue" strings - since these are not links anymore, that text is meaningless now
| To load data into CockroachDB, we recommend that you use an [`IMPORT`](import.html). If you must use a `COPY` statement, you can issue the statement from the [`psql` client](https://www.postgresql.org/docs/current/app-psql.html) command provided with PostgreSQL, or from another third-party client. | ||
|
|
||
| [Tracking GitHub Issue](https://github.com/cockroachdb/cockroach/issues/16392) No newline at end of file | ||
| Tracking GitHub Issue No newline at end of file |
Contributor
There was a problem hiding this comment.
this is not sufficient - the dangling text "tracking github issue" is not appropriate
please remove all of these empty non-link "Tracking github issue" strings
Remove all orphaned "Tracking GitHub Issue" references that were left behind after link removal. These include standalone lines, end-of-line suffixes, and "See the GitHub tracking issue for details" lines across 35 known-limitations files in v20.2, v23.1, and v23.2. Addresses review feedback from @rmloveland. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
Author
|
All review comments have been addressed:
All changes are across |
rmloveland
approved these changes
May 22, 2026
Contributor
rmloveland
left a comment
rmloveland
left a comment
There was a problem hiding this comment.
LGTM thanks @ebembi-crdb !!!
Contributor
|
@ebembi-crdb before merging this can you please make sure to port it forward to v26.3 as well? looks like #23371 was merged in the meantime |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
[#96398](https://github.com/cockroachdb/cockroach/issues/96398)to plain#96398known-limitations/includesThis is PR 2 of 10 splitting #23256 into reviewable chunks.
Jira: DOC-17056
Test plan
known-limitationincludes🤖 Generated with Claude Code